Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add ability to define composer update channel #571

Merged
merged 1 commit into from
Apr 19, 2020

Conversation

CyberLine
Copy link
Contributor

@CyberLine CyberLine commented Apr 9, 2020

Pull Request (PR) description

Yesterday, composer released 2.0 snapshot.
So with current default behaviour, if i only set

class { '::php::composer': max_age => 2, }

every composer installation is updated to version 2.0 snapshot.

So one solution is to set the 'source' to 'https://getcomposer.org/composer-stable.phar' initialy if i dont have it installed or use the param from this PR for updates, which then defaults update channel to 'stable'

@CyberLine
Copy link
Contributor Author

Since Composer 1.10.2 today, --1 and --2 are also possible channel names.

@c33s
Copy link
Member

c33s commented Apr 9, 2020

as far as i can see there are some changed lines which contain only whitespace changes. can you please ensure that there are only clean diffs of lines you really changed. this helps with the review.

manifests/composer.pp Outdated Show resolved Hide resolved
@bastelfreak bastelfreak added enhancement New feature or request needs-work not ready to merge just yet tests-fail labels Apr 10, 2020
@CyberLine
Copy link
Contributor Author

@bastelfreak
Copy link
Member

@CyberLine please rebase after #513 got merged

@CyberLine CyberLine requested a review from bastelfreak April 15, 2020 07:03
@bastelfreak bastelfreak removed needs-work not ready to merge just yet tests-fail labels Apr 19, 2020
manifests/composer.pp Outdated Show resolved Hide resolved
@bastelfreak bastelfreak merged commit ae61881 into voxpupuli:master Apr 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants